Skip to content

feat: add labels input to configure - #182

Open
NotFounds wants to merge 4 commits into
changesets:mainfrom
NotFounds:feat-enable-to-add-labels
Open

feat: add labels input to configure#182
NotFounds wants to merge 4 commits into
changesets:mainfrom
NotFounds:feat-enable-to-add-labels

Conversation

@NotFounds

@NotFounds NotFounds commented May 10, 2022

Copy link
Copy Markdown

It would be cool to attach labels when the action creates a release PR.
For example, if we add a release label to PRs, we can find release PRs easily.

I implemented labels input.
labels allows input, and we can specify comma-separated labels(include space).

e.g.

      - name: Create Release Pull Request
        uses: notfounds/changesets-action@main
        with:
          publish: yarn changeset publish
          labels: "release,test label,test-label2"

cf. https://github.com/NotFounds/changesets-monorepo/blob/d42f07d8b7def90913c71be14e9d69c8d3254876/.github/workflows/release.yml

I confirmed it works well in NotFounds/changesets-monorepo#5.
image

@changeset-bot

changeset-bot Bot commented May 10, 2022

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a614ab5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@changesets/action Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@NotFounds

Copy link
Copy Markdown
Author

@Andarist
Hi, Could you review this or assign a reviewer, please?

@alstn2468 alstn2468 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jimmynXD

jimmynXD commented Aug 2, 2022

Copy link
Copy Markdown

is this available yet????

@Andarist

Andarist commented Aug 2, 2022

Copy link
Copy Markdown
Member

I think this would be better as a separate action that could label PRs based on the input. This way you could provide the output of this action to that labeling action and we could include an example like that in our docs. I don't see a strong reason to include this new input here when this is easily achievable through composition.

@Andarist

Andarist commented Aug 2, 2022

Copy link
Copy Markdown
Member

I've created a PR here that should help to solve this need through composition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants